Skip to content

TRAC-934: feat - implement pagination for catalyst logs query command#3124

Merged
parthshahp merged 1 commit into
canaryfrom
trac-934-catalyst-query-command
Jul 24, 2026
Merged

TRAC-934: feat - implement pagination for catalyst logs query command#3124
parthshahp merged 1 commit into
canaryfrom
trac-934-catalyst-query-command

Conversation

@parthshahp

@parthshahp parthshahp commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

What/Why?

Adding the ability to paginate for the catalyst logs query command.

Testing

Tested via manually deploying a site and using the Catalyst CLI.

Screen.Recording.2026-07-22.at.8.40.14.AM.mov

Migration

No migration needed.

@changeset-bot

changeset-bot Bot commented Jul 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: d59912d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel

vercel Bot commented Jul 21, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Jul 24, 2026 1:48pm

Request Review

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Bundle Size Report

Comparing against baseline from 91427b5 (2026-07-24).

No bundle size changes detected.

@github-actions

github-actions Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Unlighthouse Performance Comparison — Vercel

Comparing PR preview deployment Unlighthouse scores vs production Unlighthouse scores.

Summary Score

Aggregate score across all categories as reported by Unlighthouse.

Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Score 90 92 92 94

Category Scores

Category Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Performance 75 73 76 98
Accessibility 95 95 95 98
Best Practices 100 100 100 100
SEO 88 88 100 100

Core Web Vitals

Metric Prod Desktop Prod Mobile Preview Desktop Preview Mobile
LCP 3.7 s 4.3 s 3.7 s 2.3 s
CLS 0.05 0.238 0.001 0
FCP 1.2 s 1.2 s 1.2 s 1.2 s
TBT 0 ms 70 ms 0 ms 0 ms
Max Potential FID 40 ms 120 ms 40 ms 40 ms
Time to Interactive 3.7 s 4.3 s 3.7 s 2.3 s

Full Unlighthouse report →

@parthshahp
parthshahp marked this pull request as ready for review July 22, 2026 12:42
@parthshahp
parthshahp requested a review from a team as a code owner July 22, 2026 12:42
const TELEMETRY_KEY_ENABLED = 'telemetry.enabled';
const TELEMETRY_KEY_ID = `telemetry.anonymousId`;

// CLI telemetry is best-effort: command completion must never be delayed by a

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was causing problems when testing, let me know if I should remove this.

@parthshahp
parthshahp force-pushed the trac-934-catalyst-query-command branch from 88f6dd2 to e7ca7bb Compare July 22, 2026 15:25
@parthshahp
parthshahp changed the base branch from alpha to canary July 22, 2026 15:25
Comment thread packages/catalyst/src/cli/lib/project.ts Outdated
@parthshahp
parthshahp force-pushed the trac-934-catalyst-query-command branch from c58c266 to d59912d Compare July 24, 2026 13:47
@parthshahp
parthshahp added this pull request to the merge queue Jul 24, 2026
Merged via the queue into canary with commit bb04eca Jul 24, 2026
17 of 18 checks passed
@parthshahp
parthshahp deleted the trac-934-catalyst-query-command branch July 24, 2026 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants